Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Single Cell Tutorial in R #4141

Merged

Conversation

Camila-goclowski
Copy link
Contributor

Built Filter, Plot, and Explore tutorial in RMarkdown using Seurat in R. First contribution to the GTN.

@Camila-goclowski Camila-goclowski requested a review from a team as a code owner May 17, 2023 14:13
@Camila-goclowski Camila-goclowski marked this pull request as draft May 17, 2023 14:14
@pavanvidem
Copy link
Member

wow! the first Seurat-based tutorial on GTN. Thanks! I am happy to review it once you're done with the draft.

@hexylena
Copy link
Member

Hi @Camila-goclowski not sure how I missed this one. Very exciting, I will make some formatting comment.s

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small formatting comments but amazing, another R tutorial!!

@shiltemann
Copy link
Member

shiltemann commented Jun 19, 2023

nice! don't forget to add yourself to our CONTRIBUTORS.yaml file :) (should also fix the linting error and allows you to get a preview of the website locally or on GitPod)

Camila-goclowski and others added 15 commits August 23, 2023 16:10
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
…tudio-WIP/tutorial.md

Co-authored-by: Pavankumar Videm <[email protected]>
@nomadscientist
Copy link
Collaborator

Dangit why is this still not working?

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to merge this and fix the build if it fails when I can

@hexylena hexylena merged commit 0c9d252 into galaxyproject:main Oct 2, 2023
2 of 3 checks passed
@hexylena
Copy link
Member

hexylena commented Oct 2, 2023

Ok, opened a new PR based off of main after this was merged and, things work fine. I suspect it was a branch specific issue, it might've been solved by updating this PR to be rebased against the latest main branch but that's also might not have been worth it.

So, yay, PR merged, congrats @Camila-goclowski on your first tutorial! Welcome to the GTN

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants